Skip to content

feat(git-commit): ryoppippi版のreferences/とrevertability哲学を取り込みフッターを中立化#154

Merged
nkoji21 merged 5 commits into
mainfrom
feat/git-commit-skill-references
Jun 15, 2026
Merged

feat(git-commit): ryoppippi版のreferences/とrevertability哲学を取り込みフッターを中立化#154
nkoji21 merged 5 commits into
mainfrom
feat/git-commit-skill-references

Conversation

@nkoji21

@nkoji21 nkoji21 commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Summary

ryoppippi さんの commit スキルが良かったので、その良い所を既存の
git-commit スキルに取り込んだ。そのまま置き換えると独自の利点
--path 引数・起動時の動的 status 埋め込み・git-commit-ja 連携)が
失われるため、マージ方式を採用した。

Changes

  • git-commit: revertability を軸にしたコミット哲学を追記
    (tiny commit 推奨、move/rename は両側を1コミット、PR で安易に
    --amend しない)。詳細例・git apply・push の手順を references/
    に分離し本体を薄く保つ。
  • references/push.md は fish 構文を zsh/sh 互換に書き換えた。
  • 共有スキル(Claude / Codex / Cursor 兼用)で嘘の co-author が
    入らないよう、Co-Authored-By 等のエージェント識別フッターを廃止。
  • git-commit-ja: 薄い委譲ラッパーだと委譲先の英語/UK スペル指示と
    衝突して推論が揺れるため、type/scope は英語・description は日本語
    という言語ルールを委譲先より優先する旨を自己完結で明記。

Notes

.cursor/settings.json の変更は今回の作業と無関係なため含めていない。

nkoji21 added 5 commits June 15, 2026 14:16
ryoppippiのcommitスキルからrevertable-commits/git-apply/pushの詳細をreferences/に分離し本体を薄く保つ。共有スキルのためCo-Authored-By等のエージェント識別フッターは付けない方針に変更。
薄いラッパーだと/git-commitの英語/UKスペル指示と日本語指示が衝突し推論が揺れるため、type/scopeは英語・descriptionは日本語という言語ルールを委譲先より優先する旨を自己完結で明記。
echoのみで停止せず保護ブランチにpushしうる不具合を修正。--push時にコマンドがフォールスルーして main/master へ push する事故を防ぐ。--path併用時のcd前置も明示。
全例を--cached既定にしてworktree誤改変を防止。context mismatchに誤った--ignore-whitespaceではなく--3way/-Cを案内するよう訂正。
git diff(引数なし)はstaged hunkや新規ファイルを取りこぼすためgit diff HEADに統一。body方針を「Why/revertability説明は許容」と緩めGood例との矛盾を解消。git-commit-jaにreferencesの実パスを明示。
@nkoji21 nkoji21 marked this pull request as ready for review June 15, 2026 05:30
@nkoji21 nkoji21 merged commit aa1b6e4 into main Jun 15, 2026
3 checks passed
@nkoji21 nkoji21 deleted the feat/git-commit-skill-references branch June 15, 2026 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant